New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ensure 2 digits for volumeHex (#71) (again) #142

Merged

jupe merged 3 commits into jupe:master from wweich:correct_volume

Sep 8, 2022

Contributor

wweich commented Apr 16, 2022 •

edited

Loading

Status

READY

Migrations

NO

Description

Fix setting of volume below 16

Related PR/Issues


          ensure 2 digits for volumeHex (#71) (again)

804d547

wweich requested a review from jupe as a code owner

April 16, 2022 08:31

coveralls commented Apr 16, 2022 •

edited

Loading

Pull Request Test Coverage Report for Build e39d2171-bc6d-497c-8e4a-1af74cf3b68a

3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.562%

Totals
Change from base Build 50ea3d80-1258-4129-a586-77927a577768:	0.0%
Covered Lines:	258
Relevant Lines:	494

💛 - Coveralls

wweich added 2 commits

April 16, 2022 10:50


          fix return for setVolume (#73) (again)

996d6a3


          fix test

1a217de

sonarqubecloud bot commented Apr 16, 2022

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

jupe merged commit 8c09c23 into jupe:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet